feat(common): updated the ed25519 dalek crate - #3473
Draft
damrobi wants to merge 1 commit into
Draft
Conversation
…ix rand_core mismatch
damrobi
temporarily deployed
to
testing-2-preview
August 5, 2026 16:56 — with
GitHub Actions
Inactive
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Content
This PR includes an update of the ed25519 dalek crate from 2.2.0 to 3.0.0. It also includes a bridge to link two incompatible version of
rand_core.Update options
Since we need to deal with two incompatible versions of
rand_core(0.6.4 and 0.10.0) we need a way to handle the code that breaks with the update to the new dalek crate.Currently only two spots break in the library due to calling
ed25519-dalek::SigningKey::generate():We have two solutions to fix it for now:
rand_core: currently implemented in commit 608e1afed25519-dalek::SigningKey::generate()to replace them with the content of thegeneratefunction to bypass the trait difference issue. This would mean the function is not following potential updates in theed25519-dalekcrateThe
ed25519-dalek::SigningKey::generate()function:Pre-submit checklist
Comments
Issue(s)
Closes #3471